Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: correct indent calculation for shiftwidth=0 #116

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

xlucn
Copy link

@xlucn xlucn commented Aug 28, 2024

Addressing some cases in #45. Fixed the logic for calculating indentation in format.lua. Previously in the case of shiftwidth=0, the indent was not using the value of tabstop, leading to incorrect indentation.

Fixed the logic for calculating indentation in format.lua. Previously in
the case of shiftwidth=0, the indent was not using the value of tabstop,
leading to incorrect indentation.
zhimsel added a commit to zhimsel/dotfiles that referenced this pull request Sep 3, 2024
This should be reverted when
zbirenbaum/copilot-cmp#116 is merged
@zhimsel
Copy link

zhimsel commented Sep 3, 2024

Just tested this PR with shiftwidth=0 and it works as expected. The indentation is no longer removed.

Thanks @xlucn!

haapmik added a commit to haapmik/copilot-cmp that referenced this pull request Nov 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants